-
Notifications
You must be signed in to change notification settings - Fork 24.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DOCS] Add JS client helpers links to docs #55216
Conversation
Pinging @elastic/es-docs (>docs) |
Pinging @elastic/es-distributed (:Distributed/CRUD) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding this @delvedor.
I left some suggestions so we avoid using /current/
links, which can often cause headaches with broken links. Using the {jsclient}
attribute should do something similar while fixing the CI tests.
Do you plan to backport these changes?
Hey @jrodewig! Sorry for the delay! |
@elasticmachine update branch |
@jrodewig all good on my side! :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I updated the links to use the {jsclient}
attribute per my earlier comment.
Thanks @delvedor!
Adds links for the Elasticsearch-js client to the bulk and scroll docs. Co-authored-by: Tomas Della Vedova <[email protected]>
Adds links for the Elasticsearch-js client to the bulk and scroll docs. Co-authored-by: Tomas Della Vedova <[email protected]>
Adds links for the Elasticsearch-js client to the bulk and scroll docs. Co-authored-by: Tomas Della Vedova <[email protected]>
This pr adds the link of the Node.js client helpers to the doc.
The doc build will likely fail because the helpers will be released in 7.7, which is still in RC.